-
Notifications
You must be signed in to change notification settings - Fork 222
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Add Python Poetry documentation #12158
Conversation
We now have native `poetry` support to manage virtual enviroments and dependencies. Ref: pulumi/pulumi#15937
{{% choosable os macos %}} | ||
|
||
```bash | ||
$ poetry add ${PACKAGE_NAME} |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I don't have a concrete package name to use here, are we ok with using syntax like ${PACKAGE_NAME}
to indicate that this is a place holder?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I think this is fine, yeah, although we don't have a particular standard around this yet (and probably should). I'll poll the team and see what they think, but IMO this is good for now.
Your site preview for commit c285045 is ready! 🎉 http://www-testing-pulumi-docs-origin-pr-12158-c2850456.s3-website.us-west-2.amazonaws.com. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Looks great! Nothing need block, just a few minor suggestions for improvement.
{{% choosable os macos %}} | ||
|
||
```bash | ||
$ poetry add ${PACKAGE_NAME} |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I think this is fine, yeah, although we don't have a particular standard around this yet (and probably should). I'll poll the team and see what they think, but IMO this is good for now.
Co-authored-by: Christian Nunciato <[email protected]>
Co-authored-by: Christian Nunciato <[email protected]>
Your site preview for commit 7fa66f0 is ready! 🎉 http://www-testing-pulumi-docs-origin-pr-12158-7fa66f05.s3-website.us-west-2.amazonaws.com. |
Proposed changes
We now have native
poetry
support to manage virtual enviroments anddependencies.
Related issues (optional)
Ref: pulumi/pulumi#15937